Move the widgets above the hpaned to their own box

This way we can re-create the contents of that box easily without disrupting
the rest of the file chooser.

Signed-off-by: Federico Mena Quintero <federico@gnome.org>

Make operation_mode_set() handle SEARCH and RECENT modes as well

Signed-off-by: Federico Mena Quintero <federico@gnome.org>

Centralize changing the operation mode in operation_mode_set()

This will be the central function to switch operation modes:  stop searches or the recent-files
process; switch widgets, etc.

We factor out the common code from recent_switch_to_browse_mode() and search_switch_to_browse_mode(),
and remove those functions.  All the code that switched modes by hand now calls
the central operation_mode_set().

Signed-off-by: Federico Mena Quintero <federico@gnome.org>

Remove duplicated code from *_switch_to_browse_mode()

That code indeed lives in operation_mode_stop(), so use it instead of
having inlined duplicates.

Signed-off-by: Federico Mena Quintero <federico@gnome.org>

Rename stop_operation() to operation_mode_stop()

We'll now prefix functions to change the impl->operation_mode with 'operation_mode',
for clarity.

Signed-off-by: Federico Mena Quintero <federico@gnome.org>

Save selection to recent-files in the asynchronous Save cases

Various paths in SAVE or CREATE_FOLDER return from ::should_respond() but
leave an asynchronous process running.  This process checks some things
in the user's selection, for example, 'does the file exist, and if so
do we need to bring up an overwrite-confirmation dialog?'.  When these
async processes complete *and* it is indeed time for the dialog to
be terminated (via the response-requested signal), we also need to
save the selection to the recently-used list - as ::should_respond() does
by itself in the cases when it can request a response immediately.

Signed-off-by: Federico Mena Quintero <federico@gnome.org>

Log to recent-files when confirming the file chooser

To make life easier for users, when apps don't properly update the recently-used list
after choosing a file, we now do that directly from the file chooser.

Signed-off-by: Federico Mena Quintero <federico@gnome.org>

Update the docs with the policies for Save dialogs

Basically, don't ever set the current folder, and only use
gtk_file_chooser_set_filename() for 'File/Save As'.  This is so
that the file chooser will be able to present its recently-used
lists as appropriate, giving the user good suggestions by default.

Signed-off-by: Federico Mena Quintero <federico@gnome.org>

No need to highlight the file's basename when setting the base folder

None of the cases where _gtk_file_chooser_entry_set_base_folder() appear to require
the entry highlighting the file's basename.  Doing the highlighting actually makes
things look weird in Save/Recent mode if you

  1. type a filename
  2. click on a recent-folder,

as right after (2) your filename would get its basename highlighted for
no apparent reason.

Signed-off-by: Federico Mena Quintero <federico@gnome.org>

Hide the Create Folder button in recent-files mode when the pathbar is on

The create-folder machinery doesn't handle that case yet; we may enable it later
once we figure out the implications for the GUI.

Signed-off-by: Federico Mena Quintero <federico@gnome.org>

Centralize the setting of the pathbar's widgets

It used to be that every part of the file chooser's code would show/hide the widgets
near the pathbar as needed.  Now we have two central functions:

  path_bar_update()
  path_bar_set_mode()

These take care of all the widget shuffling;  setting the visibility of the
pathbar, info bar, and Create Folder button as appropriate; setting the contents
of the info bar, etc. - based on the current operation_mode and action.

Signed-off-by: Federico Mena Quintero <federico@gnome.org>

Create the recently-used widgets and infobar from the pathbar widgets as well

Signed-off-by: Federico Mena Quintero <federico@gnome.org>

Move the pathbar creation to its own function

We will centralize the place where all the pathbar-related widgets are created:
the location button, the pathbar itself, the Create Folder button, and in
subsequent commits, the info bar as well.  We will deal with the pathbar/infobar
as a unit, instead of swapping them in and out in an ad-hoc fashion.

Signed-off-by: Federico Mena Quintero <federico@gnome.org>

Warn the user when he still needs to type a filename or choose a folder

Signed-off-by: Federico Mena Quintero <federico@gnome.org>

Return the recent-folder plus filename in Save mode from get_files()

Since the GtkFileChooserEntry already gets the recent-folder set upon it when a recent-folder
is selected, it already can give us the correct fully-formed path.

Signed-off-by: Federico Mena Quintero <federico@gnome.org>

Allow resolving absolute paths even if there is no base_folder

This lets us do proper completion in GtkFileChooserEntry even when no base folder
has been set.  Completion for relative paths won't work, as usual, as expected.

Set the filename entry's base folder when a recent-folder is selected

This lets the filename entry do completion relative to the selected recent-folder.

Signed-off-by: Federico Mena Quintero <federico@gnome.org>

Instruct the user to pick a folder when nothing is selected in the recent-folders list

Signed-off-by: Federico Mena Quintero <federico@gnome.org>

Put recently-used folders in the recently-used list

Signed-off-by: Federico Mena Quintero <federico@gnome.org>

Turn a struct field into a local variable

Signed-off-by: Federico Mena Quintero <federico@gnome.org>

Remove unused struct field

Signed-off-by: Federico Mena Quintero <federico@gnome.org>

Actually highlight the 'recently used' and 'search' items in the shortcuts bar

They weren't being selected in the shortcuts bar when those modes were
activated programmatically, instead of through the user selecting
them from the user interface.

Start in recently-used mode when no folder is set

In RELOAD_EMPTY mode, when no folder has been selected by the calling app, we now
start showing the recently-used list.  The rationale is as follows:

  - In Open mode, the user is likely to pick a file he has used recently.

  - In Save mode, the user is likely to want a destination folder which
    he has used recently.

For the Save case, where we want to present the user with recent folders instead
of recent files, we will make the recent-list do so in subsequent commits.

Signed-off-by: Federico Mena Quintero <federico@gnome.org>

Put the pathbar in the 'Save in folder:' row in Save mode

Now we reparent the browse_path_bar_hbox to that spot in Save mode,
or to be above the file lists in Open mode.  The pathbar makes for a very
clear indication of the location to save in.

Signed-off-by: Federico Mena Quintero <federico@gnome.org>

Get rid of the save_folder_combo

And with this we get rid of the craziness of having a separate filter model
for the combobox's model.

Signed-off-by: Federico Mena Quintero <federico@gnome.org>

Remove the expander in Save mode, and the configuration key

This effectively makes the file chooser always be in 'expanded' mode.
Later, we'll move the pathbar to the 'Save in folder:' line.

Signed-off-by: Federico Mena Quintero <federico@gnome.org>

Utility function to extract folders from the recently-used list

This extracts the parent folders from the items in the recently-used
list.  We'll use it in the file chooser to present a list of
recently-used folders.

Signed-off-by: Federico Mena Quintero <federico@gnome.org>

toolbar: don't leak the sibling path

toolbar: don't count invisible widgets when building the sibling path

Also, fix RTL nth-child handling for toolbars.

toolbar: invalidate order when an item visibility changes

Test box and toolbar regions in parallel

This makes it easy to see discrepancies.
Plus, it is fun

toolbar: nth-child support for GtkToolbar

This doesn't quite work right yet.

box: follow children visibility when building the sibling path

This fixes nth-child not working for widgets which were not
visible before being added to the box, but it's also a better fix
for https://bugzilla.gnome.org/show_bug.cgi?id=652769

Revert "gtkbox: Invalidate order on changes, even if the child is not visible"

This reverts commit 8cb0cc58e321641ebda699b9cbdc0067aeb56d3f.

Fix up docs for predefined style classes.

The table was incomplete and out of date. Instead, just
put a list of links in that place, and move all the extra
documentation to the macros. Bug 653785

GdkKeymap: Prevent an infinite loop in the non-XKB case

Somehow the increment got lost, as comparison with the 2.x code
shows.

gtkbox: Invalidate order on changes, even if the child is not visible

When we build the sibling path for the order, we do not skip hidden
children (since, quoting the comment, "we cannot reliably detect changes
in widget visibility"). So we need to invalidate the order when hidden
children are reordered and removed.

https://bugzilla.gnome.org/show_bug.cgi?id=652769

[gi] add more annotations to force drag and drop api to be methods

Updated Dutch translation by Wouter Bolsterlee

[gi] make sure gtk_drag_dest_unset is annotated as a method

Make GtkBuilder do not delay construct properties that can be resolved
(so construct is respected when possible) and skip construct only props that
can not.

gtk/gtkmenuitem: Remove reference to deprecated and removed GtkItem

Update VS2010 READNE.txt

This file should have Windows CRLF EOL as well

Re-attempt to correct EOL on VS2010 solution

This time I realized that I needed to set autocrlf=false on my Windows side
... ugh...

This is one of those files that must have CRLF line endings to work
correctly :|

Use a PAGE_STEP macro instead of a magic number

Helps to understand the code.

https://bugzilla.gnome.org/show_bug.cgi?id=653512

Updated Belarusian translation.

Don't forget to initialize a variable

Pointed out by clang in bug 653333.

[gi] correct annotation for GtkMenuPositionFunc callbacks

bgo#653191 - fix uninitialized variable in gtk_file_chooser_entry.c

Speed up page insertion

Make sure to call widget_set_child_visible(false) on all tabs except the
current before calling widget_set_parent.

Updated Belarusian translation.

Updated Belarusian translation.

Update VS projects distribution

Change the EXTRA_DIST to distribute the gtk3-demo projects (instead of
gtk-demo)

Add headers required by gtk/gtkwin32embedwidget.c

Some functions from the added headers are needed by this file to compile
without C4013 (a.k.a implicit declaration of ...) warnings/errors

Update VS project files

-Reinstate build/win32/vs10/gtk+.sln with the correct EOL (DOS/Windows), so
 that it will be correctly recognized by Windows instead of having the
 annoying "Unrecognized Visual Studio Version".

-Update property sheets to reflect on new headers added

-Change the demo program to be gtk3-demo.exe, to be consistent with the
 names on other platforms, and updated/renamed related project/solution
 files, and added overlay.c to the list of demo sources.

Temporarily remove build/win32/vs10/gtk+.sln

The line endings need to be DOS/Windows...

GtkPaned docs: Remove sentence saying see GtkPaned.

Updated Persian Translation

Track wayland protocol changes.

introspection.m4: Update from g-i

This fixes substitution with dash.

Updated Galician translations

Avoid GApplication being released twice on gtk_application_remove_window() calls

Removing the window from the window list before setting the
application to %NULL avoids gtk_application_remove_window() triggering
another call to gtk_application_window_removed(), which would release
the application a second time.

https://bugzilla.gnome.org/show_bug.cgi?id=653053

docs: remove tmpl directories

We do not use template files anymore

configure.ac: Generate xz tarballs with ustar format by default

Update VS property sheets

As Cairo and Cairo-GObject are often built as two seperate DLLs/modules,
set the property sheets to link to both libraries, instead of using the
previous approach where a monolithic Cairo DLL which contains GObject
support is used.

Refine Visual C++ 2010 projects

-Remove unneeded tags from projects
-Seperate intermediate directories for projects to avoid rebuilding/
 linking on every rebuild and MSBuild errors (et al.) for not being
 able to write into build log files as they are in use.

Updated Norwegian bokmål translation.

Updated Spanish translation

Add sibling information to widget path string representations

Fix css syntax in example

Remove an unused variable

Remove nonexisting properties

glade insists on adding GtkGrid:n-rows and :n-columns, for
no good reason...

tests: import the widget-factory test for theming

Courtesy of Andrea Cimitan <andrea.cimitan@canonical.com>.

Update keybinding docs

These were still explaining gtkrc syntax.

Remove unnecessary includes

gtkstyle.h is similarly unused.

Remove unused includes

Nothing is using gtkrc.h functionality any more.

Silence a warning

GtkBox: don't reset styles too often

info_bar: call style_updated at the end of info_bar_init

This fix a render bug where the infobar would not add borders
arount its content. Fix proposed by Benjamin Otte.

Update Visual C++ projects

Use G_ENABLE_DEBUG for all debug builds

toolbar: Better fix for animations

It turns out there's more places where the toolbar item size is used as
the margin box instead of the content box. Because of that, store the
margin box when allocating and use it whenever calls
toolbar_content_get_allocation() instead of calling
gtk_widget_get_allocation().

toolbar: Handle the fact that size_allocate() != get_allocation()

size_allocate() allocates the available space for the margin box,
get_allocation() returns the actual space of the content box and those
can be different. And then animations never stop.
If that makes you go "huh?", you might want to read
http://www.w3.org/TR/CSS21/box.html
and the docs for gtk_widget_compute_align().

reftests: fix background-area reftest after the last commit

themingengine: shrink the background size to the padding-box

This fixes a regression introduced by this commit [1] that causes
background to draw beyond the padding-box, which breaks rounded corners,
(in addition to not being compliant with what CSS does by default).

[1] http://git.gnome.org/browse/gtk+/commit/?id=c56700ca3029a9ab1362009bff35f2a70e432c02

https://bugzilla.gnome.org/show_bug.cgi?id=652618

themingengine: plug a couple of leaks

border-image: Fix leakage

tests: Add tests for font-family

This readds the tests now that font-family works again. They were
previously removed when font-family support was backed out due to bugs.

css: Make font property a shorthand

... and implement the CSS font properties:
- font-size
- font-style
- font-family
- font-weight
- font-variant

This is the second try at this. The first was backed out previously due
to bugginess. Let's hope this one survives a bit longer.

Also makes the font-family CSS test work again.

Add a missing break statement

Add a missing break statement

Avoid an oob read

Remove a redundant NULL check

Avoid a dereference-before-null-check

Give this code a chance to work

At least it does not get uninitialized variable warnings now.

Make this code more meaningful

Don't forget to cleanup varargs

tests: Add simple test for image clipboard

Updated Slovenian translation

image: Initialize variable

GtkAssistant: improve button labels

Forward/Back/Last has been declared strange; so go with something
a little more friendly: Continue/Go Back/Finish.
https://bugzilla.gnome.org/show_bug.cgi?id=652506